Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bump jsoncons to 0.172.0 #1911

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Bump jsoncons to 0.172.0 #1911

merged 1 commit into from
Nov 27, 2023

Conversation

aleksraiden
Copy link
Contributor

@aleksraiden aleksraiden commented Nov 27, 2023 •
edited
Loading

Bump jsoncons to 0.172.0. Full release notes - https://github.com/danielaparker/jsoncons/releases/tag/v0.172.0

Key features

  • Fixed issue endless loop in jmespath::search danielaparker/jsoncons#469 affecting JSMESPath expressions with terminating CR
  • Added result_option sort_descending
  • Added new classes basic_json_location, basic_path_element, and basic_path_node
  • Added a new functions get and remove for selecting a single JSON value from a JSON document at
    a location represented by a json_location

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Nov 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
3.0% 3.0% Duplication

@mapleFU
Copy link
Member

mapleFU commented Nov 27, 2023

Aha I've a requirement fixed by jsoncons: danielaparker/jsoncons#466

Let's checkin this

@PragmaTwice
Copy link
Member

Another new feature here danielaparker/jsoncons#467 (comment).

cc @Qiaolin-Yu

@git-hulk git-hulk merged commit 963839d into apache:unstable Nov 27, 2023
30 checks passed
@Qiaolin-Yu Qiaolin-Yu mentioned this pull request Nov 27, 2023
Merged
@aleksraiden aleksraiden deleted the aleksraiden-patch-4 branch February 13, 2024 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mapleFU mapleFU mapleFU approved these changes

@git-hulk git-hulk git-hulk approved these changes

@torwig torwig Awaiting requested review from torwig

@PragmaTwice PragmaTwice Awaiting requested review from PragmaTwice

@shangxiaoxiong shangxiaoxiong Awaiting requested review from shangxiaoxiong

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aleksraiden @mapleFU @PragmaTwice @git-hulk